Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: provision to disable frappe.io auth dependency #1750

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

saurabh6790
Copy link
Member

No description provided.

@phot0n
Copy link

phot0n commented May 20, 2024

Don't raise exception if the url is not found instead of adding another field?
Better UX wise with a new field :p

Also all this logic which connects and does stuff on frappe.io and erpnext.com should probably be moved to another app?

@adityahase adityahase requested a review from BreadGenie as a code owner June 6, 2024 05:38
@saurabh6790 saurabh6790 force-pushed the frappe-io-auth branch 2 times, most recently from 34170c7 to fca8687 Compare June 26, 2024 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants